Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Agrega validaciones a el formulario de solicitud de préstamo #401

Open
wants to merge 4 commits into
base: development
Choose a base branch
from

Conversation

TonyMckes
Copy link
Member

@TonyMckes TonyMckes commented Apr 25, 2024

Issues relacionados.

Que implementa o corrige este Pull Request.

Se agregan validaciones (server side) con la libreria zod a el formulario de solicitud de préstamo para mostrar mensajes de error apropiadamente

Tipo de Pull Request:

  • Refactor (Modificación de código existente).
  • New feature (Introducción de nueva funcionalidad).
  • Bug fix (Corrección de error/es)
  • Performance (Mejoras de optimización).
  • Docs (Actualización/extensión de documentación).

Pasos necesarios para probar/visualizar los cambios implementados:

Probar en prod:

  1. Dirigirse a https://juguetear-web-git-feat-form-validations-juguetear.vercel.app/juguete/dinosaurio-dragon (o algun juguete disponible)
  2. Probar el formulario

Probar localmente (dev server):

  1. Instalar dependencias yarn install
  2. Ejecutar el commando de build yarn build
  3. Iniciar la aplicacion en modo prod yarn start
  4. Dirigirse a http://127.0.0.1:3000/juguete/dinosaurio-dragon (o algun juguete disponible)
  5. Probar el formulario

Capturas de pantalla/videos:

  • Nuevo componente.
  • Modificación a elemento en la interfaz de usuario.
  • No aplica.

image


Código de conducta y contribuir.

⚠️ Antes de realizar el Pull Request, por favor asegúrate de haber leído el código de conducta y cómo contribuir.

Note

⛑️ En caso de que necesites ayuda o tengas alguna duda recordá que también podés preguntar en el canal #juguetear dentro de la comunidad FrontendCafé en Discord. El código de conducta de este proyecto es extensible también a tu participación en el server de FrontendCafé.

@TonyMckes TonyMckes added the type: feature New feature or request label Apr 25, 2024
@TonyMckes TonyMckes self-assigned this Apr 25, 2024
Copy link

vercel bot commented Apr 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
juguetear-web ✅ Ready (Inspect) Visit Preview Apr 25, 2024 4:25pm
storybook-juguetear-web ✅ Ready (Inspect) Visit Preview Apr 25, 2024 4:25pm

Copy link

@dylanson25 dylanson25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Me parece bien como manejaste las validaciones con zod.
Lo unico que no me convence es el diseno de los mensajes de error.
Pero eso ya es mas personal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants